Skip to content

Conversation

@bquorning
Copy link
Contributor

@bquorning bquorning commented Mar 6, 2025

Let’s publish a plugin’ed version ✨

Before submitting the PR make sure the following are checked:

  • Feature branch is up-to-date with main (if not - rebase it).
  • Squashed related commits together.
  • Added tests.
  • Updated documentation.
  • Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
  • The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@bquorning bquorning marked this pull request as ready for review March 6, 2025 07:54
@bquorning bquorning requested a review from a team as a code owner March 6, 2025 07:54
@bquorning
Copy link
Contributor Author

cc @ydah

@ydah
Copy link
Member

ydah commented Mar 6, 2025

I have one more PR to add, so please wait a moment 🙇🏻

@koic
Copy link
Member

koic commented Mar 10, 2025

AFAIK, this is the last extension gem managed by the RuboCop org that has not yet been released as a plugin. With this release, users will be able to consistently use plugins for all officially maintained gems under the RuboCop org.

Having a consistent usage pattern for gems within the RuboCop org is a meaningful step forward.

@ydah
Copy link
Member

ydah commented Mar 10, 2025

@bquorning Sorry. Let's move forward with the release.

@bquorning bquorning marked this pull request as ready for review March 10, 2025 07:35
@bquorning bquorning merged commit 0e39b09 into main Mar 10, 2025
28 checks passed
@bquorning bquorning deleted the release branch March 10, 2025 07:37
@bquorning
Copy link
Contributor Author

AFAIK, this is the last extension gem managed by the RuboCop org that has not yet been released as a plugin.

Actually, rubocop-rspec_rails is missing too. I’ll open a PR.

@bquorning
Copy link
Contributor Author

rubocop-rspec_rails PR here: rubocop/rubocop-rspec_rails#66

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants